Skip to content

docs: correct S7CommPlus support guidance - #839

Open
gijzelaerr wants to merge 2 commits into
masterfrom
docs/s7commplus-support-accuracy
Open

docs: correct S7CommPlus support guidance#839
gijzelaerr wants to merge 2 commits into
masterfrom
docs/s7commplus-support-accuracy

Conversation

@gijzelaerr

@gijzelaerr gijzelaerr commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • document the legacy V1 SessionKey path instead of claiming a PUT/GET fallback
  • clarify the synchronous and asynchronous authentication boundaries
  • replace the nonexistent S7CommPlus read_tag example with the current read_symbolic API
  • distinguish classic S7 symbol/layout limitations from S7CommPlus browsing capabilities
  • correct the remaining browse() docstrings and runtime warning so they no longer advertise nonexistent APIs or fallback behavior

Related to #710, #836, and #837.

Validation

  • uv run --frozen sphinx-build -E -N -b html doc/ doc/_build -W
  • uv run --frozen pytest (1821 passed, 78 skipped)
  • uv run --frozen pre-commit run --all-files
  • uv build --no-sources

@gijzelaerr
gijzelaerr requested a review from spreeker September 1, 2026 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant